Skip to content

refactor(@angular-devkit/architect): use custom indent logger #33152

Merged
clydin merged 2 commits into
angular:mainfrom
clydin:refactor/custom-logging
May 12, 2026
Merged

refactor(@angular-devkit/architect): use custom indent logger #33152
clydin merged 2 commits into
angular:mainfrom
clydin:refactor/custom-logging

Conversation

@clydin

@clydin clydin commented May 8, 2026

Copy link
Copy Markdown
Member

Replaces createConsoleLogger with a custom logging.IndentLogger implementation to remove dependency on @angular-devkit/core/node helper.

@clydin clydin added the target: major This PR is targeted for the next major release label May 8, 2026
@clydin clydin marked this pull request as ready for review May 8, 2026 15:33

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request replaces the use of createConsoleLogger with logging.IndentLogger in both the architect and schematics CLIs. The logging logic is now manually handled via subscriptions, allowing for more direct control over output streams and styling. Additionally, the schematics CLI tests were updated to use PassThrough streams and stripVTControlCharacters for more robust output verification. I have no feedback to provide.

@clydin clydin force-pushed the refactor/custom-logging branch 3 times, most recently from ae361e7 to 4334bd3 Compare May 8, 2026 16:24
clydin added 2 commits May 12, 2026 13:39
Replaces `createConsoleLogger` with a custom `logging.IndentLogger`
implementation to remove dependency on `@angular-devkit/core/node` helper.
…d improve tests

Replaces `createConsoleLogger` with a custom `logging.IndentLogger`
implementation that respects custom output streams.

Also refactors tests to use standard `PassThrough` streams and
`stripVTControlCharacters` instead of custom mocks.
@alan-agius4 alan-agius4 force-pushed the refactor/custom-logging branch from 4334bd3 to cd08f7d Compare May 12, 2026 13:39
@clydin clydin added the action: merge The PR is ready for merge by the caretaker label May 12, 2026
@clydin clydin merged commit 653e4e9 into angular:main May 12, 2026
34 of 35 checks passed
@clydin

clydin commented May 12, 2026

Copy link
Copy Markdown
Member Author

This PR was merged into the repository. The changes were merged into the following branches:

@clydin clydin deleted the refactor/custom-logging branch May 12, 2026 14:26
@angular-automatic-lock-bot

Copy link
Copy Markdown

This pull request has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Jun 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: @angular-devkit/architect area: @angular-devkit/schematics-cli target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants